-
-
Notifications
You must be signed in to change notification settings - Fork 16k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Correct mutation adding the missing parameters #11109
Conversation
Correct mutation considering the higher number of segmentation parameters compared to object detection. Fixes ultralytics#9730 Signed-off-by: Iker Lluvia <iker.lluvia@tekniker.es>
@iker-lluvia thanks for the PR. You should not redefine dicts or lists that already exist in YOLOv5. Duplicating code increases maintenance and needlessly grows the codebase. The segmentation keys are here: yolov5/utils/segment/metrics.py Lines 180 to 200 in 85f6019
|
Sorry, I wanted to keep the changes to a minimum and the result to be similar to how the general train.py, the one that doesn't correspond to segmentation, is programmed. |
@iker-lluvia PR looks good, merged! |
* Correct mutation adding the missing parameters Correct mutation considering the higher number of segmentation parameters compared to object detection. Fixes ultralytics#9730 Signed-off-by: Iker Lluvia <iker.lluvia@tekniker.es> * Use already defined segmentation keys from segment/metrics.py --------- Signed-off-by: Iker Lluvia <iker.lluvia@tekniker.es> Co-authored-by: Glenn Jocher <glenn.jocher@ultralytics.com>
* Correct mutation adding the missing parameters Correct mutation considering the higher number of segmentation parameters compared to object detection. Fixes ultralytics#9730 Signed-off-by: Iker Lluvia <iker.lluvia@tekniker.es> * Use already defined segmentation keys from segment/metrics.py --------- Signed-off-by: Iker Lluvia <iker.lluvia@tekniker.es> Co-authored-by: Glenn Jocher <glenn.jocher@ultralytics.com>
Correct mutation considering the higher number of segmentation parameters compared to object detection.
Fixes #9730
🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Updated hyperparameters mutation logic in YOLOv5 training algorithm.
📊 Key Changes
print_mutation
from using all keys to a specific subset (keys 4 to 16).🎯 Purpose & Impact